Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix typo in promtail ClusterRole #2152

Merged
merged 1 commit into from
Jun 1, 2020
Merged

Fix typo in promtail ClusterRole #2152

merged 1 commit into from
Jun 1, 2020

Conversation

eamonryan
Copy link
Member

@eamonryan eamonryan commented May 29, 2020

pod is not a valid resource in k8s, but pods is.

What this PR does / why we need it:
The existing ClusterRole won't work for pods, since pod is not a valid resource. pods is.
Which issue(s) this PR fixes:
N/A

Special notes for your reviewer:

Checklist

  • Documentation added (it is sort of a docs fix)
  • [N/A] Tests updated (docs fix)

`pod` is not a valid resource in k8s, but `pods` is.
@CLAassistant
Copy link

CLAassistant commented May 29, 2020

CLA assistant check
All committers have signed the CLA.

@codecov-commenter
Copy link

Codecov Report

Merging #2152 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2152      +/-   ##
==========================================
- Coverage   61.50%   61.44%   -0.06%     
==========================================
  Files         149      149              
  Lines       11935    11935              
==========================================
- Hits         7341     7334       -7     
- Misses       4012     4015       +3     
- Partials      582      586       +4     
Impacted Files Coverage Δ
pkg/promtail/targets/tailer.go 73.86% <0.00%> (-4.55%) ⬇️
pkg/promtail/targets/filetarget.go 68.67% <0.00%> (-1.81%) ⬇️
pkg/logql/evaluator.go 90.90% <0.00%> (-0.56%) ⬇️
pkg/querier/queryrange/downstreamer.go 97.93% <0.00%> (+2.06%) ⬆️

Copy link
Contributor

@cyriltovena cyriltovena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cyriltovena cyriltovena merged commit a69a830 into master Jun 1, 2020
@cyriltovena cyriltovena deleted the eamonryan-patch-1 branch June 1, 2020 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants